added architecture diagram and readme#11
Conversation
dmatrix
commented
Apr 3, 2026
- added architecture diagram
- minor tweak to readme
Signed-off-by: Jules Damji <dmatrix@comcast.net>
Signed-off-by: Jules Damji <dmatrix@comcast.net>
… agents to the Supervisor Signed-off-by: Jules Damji <dmatrix@comcast.net>
There was a problem hiding this comment.
The SVG looks good and the table format for the subagents is a nice upgrade.
One note on the new "Step 1: Create a SQL Warehouse" section: I think the intent is good (helping users find their warehouse ID), but the framing could cause some confusion:
-
Most users already have a warehouse. Databricks auto-provisions a "Starter Warehouse" in every workspace, and many users will have others already. Framing this as "Create a SQL Warehouse" implies they always need to create a new one, which could lead to unnecessary warehouse proliferation.
-
Genie requires Pro or Serverless specifically — Classic warehouses won't work. Worth mentioning if we're going to walk users through this.
The original README handled this by listing "a SQL Warehouse ID" in the prerequisites and showing it in the deploy command, which worked fine. If we want to be more helpful, I'd suggest either:
- Keep it in Prerequisites with a brief note on where to find the warehouse ID (SQL Warehouses in the sidebar → click your warehouse → ID is in the URL or Connection Details tab), or
- Reframe the step as "Step 1: Find your SQL Warehouse ID" with language like "Select an existing Pro or Serverless SQL warehouse (or create one if needed)" rather than leading with creation.
Minor other things I noticed:
- "colonoies" → "colonies" in the subagent table
Signed-off-by: Jules Damji <dmatrix@comcast.net>